deepl 迁移公共构造 (fix #334) - #360
Merged
Merged
Conversation
第二家引擎接入 createByokEngine：请求格式、语言码归一化、免费版 与专业版端点区分完全不变，失败类别与文案一致；公共构造未因接入 第二家而新增特例扩展点。 Co-Authored-By: zhexuancai-uts <261878103+zhexuancai-uts@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
问题
DeepL 仍自持整段骨架（闸门 / 取 key / 分类抛错），与 #333 的公共构造重复。
根因
第二家引擎未接入公共构造，抽象 seam 未得到验证。
修复
DeepL 迁移到
createByokEngine：请求格式、语言码归一化（ZH-HANS / ZH-HANT 等）、免费版（:fx）与专业版端点区分逻辑完全不变，失败类别与错误文案与改造前一致；公共构造未因接入第二家引擎而新增任何特例扩展点。验证
pnpm typecheck通过；pnpm test647 个用例全部通过（DeepL HTTP、语言码契约、跨引擎分类契约、探测契约等既有测试未修改且全部通过）；pnpm build正常。Closes #334